Skip to content

Conversation

laurit
Copy link
Contributor

@laurit laurit commented Jan 22, 2025

and collect info about flaky tests into a google sheet.

@github-actions github-actions bot requested a review from theletterf January 22, 2025 13:54
@laurit laurit marked this pull request as ready for review January 22, 2025 17:46
@laurit laurit requested a review from a team as a code owner January 22, 2025 17:46
common:
uses: ./.github/workflows/build-common.yml
with:
max-test-retries: 0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would think if we wanted to enable retries we would need to increase this value instead of remove it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See

val defaultMaxRetries = if (System.getenv().containsKey("CI")) 5 else 0
val maxTestRetries = gradle.startParameter.projectProperties["maxTestRetries"]?.toInt() ?: defaultMaxRetries

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you for pointing that out

run: cat build-scan.txt

- name: Get current job url
uses: Tiryoh/gha-jobid-action@be260d8673c9211a84cdcf37794ebd654ba81eef # v1.4.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have security worries about using unpopular external actions, is there another way? would it be ok to just link to the run? https://github.com/$GITHUB_REPOSITORY/actions/runs/$GITHUB_RUN_ID

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replaced that action with a different solution that uses actions/github-script

Copy link
Member

@trask trask left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️

@laurit laurit merged commit 8aa2584 into open-telemetry:main Jan 24, 2025
58 checks passed
@laurit laurit deleted the flaky-test-reporter branch January 24, 2025 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants